Skip to content

Conversation

@igdmdimitrov
Copy link
Contributor

No description provided.

@igdmdimitrov igdmdimitrov changed the title feat(chat-ai): add chat ai component feat(chat): add chat component May 22, 2025
@igdmdimitrov igdmdimitrov reopened this May 22, 2025
@desig9stein
Copy link
Contributor

Issues Found in Chat Component

@SisIvanova @igdmdimitrov Those are some of the observations I have from using the component inside the Stream-Manager app

Auto-scrolling disrupts reading

Problem:
When reading older messages (scrolling up), if a new message arrives, the chat automatically scrolls to the bottom. This interrupts the user’s reading flow.

Expected behavior:
The component should only auto-scroll if the user is already at the bottom of the chat. If the user has scrolled up, the component should not auto-scroll. Instead, display a “New messages” indicator (e.g., a button or badge) that allows the user to jump to the latest message when they choose.

Empty message-header/message-attachments/message-actions containers remain in the DOM

Problem:
The component always renders the containers for message-header, message-attachments, and message-actions, even when they are empty. Because these containers are inside a flex container with gap, they create unwanted spacing and styling issues.

Expected behavior:

  • Preferred: Remove empty template containers from the DOM.
  • Alternative: Apply display: none when they are empty, so they don’t affect layout.

No customization for message container spacing

Problem: I had to override the component to remove the default padding and margin on the message container. Other developers may face the same issue if they have custom layouts.

Solution: Provide CSS custom properties (variables) for message container padding and margin. This allows consumers to adjust spacing without resorting to overrides.

@gedinakova gedinakova marked this pull request as ready for review September 18, 2025 07:21
@gedinakova gedinakova self-assigned this Sep 18, 2025
@rkaraivanov rkaraivanov merged commit 9a96545 into master Sep 18, 2025
5 checks passed
@rkaraivanov rkaraivanov deleted the dmdimitrov/chat-ai-component branch September 18, 2025 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants